Skip to content

Conversation

@Mat001
Copy link
Contributor

@Mat001 Mat001 commented Oct 31, 2025

Summary

  • Expose prediction endpoint in project config
  • can be optionally specified in config manager, the default is prediction endpoint, user can specify some other endpoint for proxy/testing purposes

Test plan

  • new unit tests

Issues

@Mat001 Mat001 marked this pull request as ready for review November 4, 2025 17:37
…-12010-py

Resolved conflict in optimizely/optimizely.py:
- Kept master's new cmab_service parameter support
- Added prediction endpoint support inside the else block (when using default CMAB service)
- All CMAB tests passing (22/22)
… github.com:optimizely/python-sdk into mpirnovar-expose-prediction-endpoint-fssdk-12010-py
@pvcraven pvcraven requested a review from Copilot November 5, 2025 15:53
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Contributor

@FarhanAnjum-opti FarhanAnjum-opti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@Mat001 Mat001 merged commit 194fa14 into master Nov 6, 2025
24 of 26 checks passed
@Mat001 Mat001 deleted the mpirnovar-expose-prediction-endpoint-fssdk-12010-py branch November 6, 2025 04:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants